From bf497ed643ee77b1439d41356445411b50ce4569 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Florian=20M=C3=BCllner?= Date: Fri, 26 Jul 2019 02:28:40 +0200 Subject: [PATCH] pointerA11yTimeout: Use notify handler instead of onUpdate callback We either need to queue a repaint on opacity updates or we don't, whether the opacity change happens in an animation or not shouldn't matter. https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/654 --- js/ui/pointerA11yTimeout.js | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/js/ui/pointerA11yTimeout.js b/js/ui/pointerA11yTimeout.js index 67b58f8af..0bbe4a46c 100644 --- a/js/ui/pointerA11yTimeout.js +++ b/js/ui/pointerA11yTimeout.js @@ -17,6 +17,8 @@ class PieTimer extends St.DrawingArea { can_focus: false, reactive: false }); + + this.connect('notify::opacity', this.queue_repaint.bind(this)); } vfunc_repaint() { @@ -69,7 +71,6 @@ class PieTimer extends St.DrawingArea { { opacity: 255, time: duration / 1000, transition: 'easeOutQuad', - onUpdate: () => this.queue_repaint(), onComplete: () => this.stop() }); } -- 2.39.5